Skip to content

build: def NODE_USE_NODE_CODE_CACHE only used in node_mksnapshot#63588

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:node_use_node_code_cache
May 28, 2026
Merged

build: def NODE_USE_NODE_CODE_CACHE only used in node_mksnapshot#63588
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:node_use_node_code_cache

Conversation

@legendecas
Copy link
Copy Markdown
Member

NODE_USE_NODE_CODE_CACHE is only used in node_mksnapshot.cc. It is unnecessary to define it for target libnode.

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels May 26, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecas legendecas force-pushed the node_use_node_code_cache branch from dc8150a to 4577d5e Compare May 26, 2026 18:32
@legendecas legendecas added request-ci Add this label to start a Jenkins CI on a PR. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels May 26, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 26, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@legendecas legendecas added the commit-queue Add this label to land a pull request using GitHub Actions. label May 28, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 28, 2026
@nodejs-github-bot nodejs-github-bot merged commit 54f7e89 into nodejs:main May 28, 2026
75 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Landed in 54f7e89

@legendecas legendecas deleted the node_use_node_code_cache branch May 28, 2026 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants